resultMembers.addGroupMembersResult - Code Metrics - Inspection of "contacts as members" - nextcloud/circles - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#120)

by Maxence

created 2017-08-25 11:49 UTC

resultMembers.addGroupMembersResult A

↳ Parent: js/circles.app.results.members.js

Complexity

Conditions	4
Paths	4

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	21
rs	9.0534
cc	4
nc	4
nop	1

1 Function

Rating	Name	Duplication	Size	Complexity
A	resultMembers.addMemberResult	0	9	1

/*
 * Circles - Bring cloud-users closer together.
 *
 * This file is licensed under the Affero General Public License version 3 or
 * later. See the COPYING file.
 *
 * @author Maxence Lange <[email protected]>
 * @copyright 2017
 * @license GNU AGPL version 3 or any later version
 *
 * This program is free software: you can redistribute it and/or modify
 * it under the terms of the GNU Affero General Public License as
 * published by the Free Software Foundation, either version 3 of the
 * License, or (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU Affero General Public License for more details.
 *
 * You should have received a copy of the GNU Affero General Public License
 * along with this program.  If not, see <http://www.gnu.org/licenses/>.
 *
 */

/** global: OC */
/** global: OCA */
/** global: Notyf */

/** global: actions */
/** global: nav */
/** global: elements */
/** global: curr */
/** global: api */


var resultMembers = {


	generateItemResult: function (search, value) {

		switch (value.type) {

			case define.typeUser:

				return resultMembers.generateItemUser(search, value);
			case define.typeGroup:
				return resultMembers.generateItemGroup(search, value);
			case define.typeContact:
				return resultMembers.generateItemContact(search, value);
		}

	},


	enhanceSearchResult: function (search, display) {
		display = escapeHTML(display);
		if (search.length > 0) {
			display = display.replace(new RegExp('(' + search + ')', 'gi'), '<b>$1</b>');
		}

		return display;
	},


	generateItemUser: function (search, value) {
		return '<div class="result_top">' +
			resultMembers.enhanceSearchResult(search, value.data.display) + '</div>' +
			'<div class="result_bot">' + t('circles', 'Local User') + '</div>';
	},

	generateItemGroup: function (search, value) {
		return '<div class="result_top">' +
			resultMembers.enhanceSearchResult(search, value.data.display) + '</div>' +
			'<div class="result_bot">' + t('circles', 'Local Group') + '</div>';
	},

	generateItemContact: function (search, value) {
		var display = resultMembers.enhanceSearchResult(search, value.data.display);
		var email = resultMembers.enhanceSearchResult(search, value.data.email);
		var org = resultMembers.enhanceSearchResult(search, value.data.organization);
		if (email !== '') {
			email = ' - ' + email;
		}

		if (org !== '') {
			display += '   (' + org + ')';
		}

		return '<div class="result_top">' + display + '</div>' +
			'<div class="result_bot">' + t('circles', 'Contact') + email + '</div>';
	},


	searchMembersResult: function (response) {

		elements.membersSearchResult.children().remove();

		if (response === null) {
			elements.membersSearchResult.fadeOut(0);
			return;
		}

		var currSearch = response.search;
		$.each(response.result, function (index, value) {
			elements.membersSearchResult.append('<div class="members_search" data-type="' +
				value.type + '" data-ident="' + escapeHTML(value.ident) + '">' +
				resultMembers.generateItemResult(currSearch, value) + '</div>');
		});

		$('.members_search').on('click', function () {
			var ident = $(this).attr('data-ident');
			var type = $(this).attr('data-type');

			if (Number(type) === define.typeGroup) {


				OC.dialogs.confirm(
					t('circles',
						'This operation will add/invite all members of the group to the circle'),
					t('circles', 'Please confirm'),
					function (e) {
						if (e === true) {
							api.addMember(curr.circle, ident, type, resultMembers.addMemberResult);
						}
					});

			} else {
				api.addMember(curr.circle, ident, type, resultMembers.addMemberResult);
			}
		});

		// elements.fillMembersSearch('users', response.ocs.data.exact.users,
		// response.ocs.data.users); elements.fillMembersSearch('groups',
		// response.ocs.data.exact.groups, response.ocs.data.groups);  if
		// (elements.membersSearchResult.children().length === 0) {
		// elements.membersSearchResult.fadeOut(0); return; }  $('.members_search').on('click',
		// function () { curr.searchUserSelected = $(this).attr('searchresult'); if
		// ($(this).attr('source') === 'groups') {  OC.dialogs.confirm( t('circles', 'This
		// operation will add/invite all members of the group to the circle'), t('circles', 'Please
		// confirm'), function (e) { if (e === true) { api.addGroupMembers(curr.circle,
		// curr.searchUserSelected, resultMembers.addGroupMembersResult); } }); } else {
		// api.addMember(curr.circle, curr.searchUserSelected, resultMembers.addMemberResult); } });
		elements.membersSearchResult.fadeIn(300);
	},


	addMemberResult: function (result) {

		console.log(JSON.stringify(result));


		resultMembers.addMemberUserResult(result);
		resultMembers.addMemberGroupResult(result);
		resultMembers.addMemberMailResult(result);
		resultMembers.addMemberContactResult(result);
	},


	addMemberUserResult: function (result) {
		if (result.user_type !== define.typeUser) {

			return;
		}

		if (curr.circleDetails.type === define.typeClosed) {
			resultMembers.inviteMemberResult(result);
			return;
		}

		if (result.status === 1) {
			OCA.notification.onSuccess(
				t('circles', "The member '{name}' was added to the circle",
					{name: result.display}));

			nav.displayMembers(result.members);
			return;
		}

		OCA.notification.onFail(
			t('circles', "The member '{name}' could not be added to the circle",
				{name: result.display}) +
			': ' + ((result.error) ? result.error : t('circles', 'no error message')));
	},


	addMemberMailResult: function (result) {
		if (result.user_type !== define.typeMail) {

			return;
		}

		if (result.status === 1) {
			OCA.notification.onSuccess(
				t('circles', "The email address '{email}' was added to the circle",
					{email: result.display}));

			nav.displayMembers(result.members);
			return;
		}
		OCA.notification.onFail(
			t('circles', "The email address '{email}' could not be added to the circle",
				{email: result.display}) +
			': ' + ((result.error) ? result.error : t('circles', 'no error message')));
	},

	addMemberContactResult: function (result) {
		if (result.user_type !== define.typeContact) {

			return;
		}

		if (result.status === 1) {
			OCA.notification.onSuccess(
				t('circles', "The contact '{contact}' was added to the circle",
					{contact: result.display}));

			nav.displayMembers(result.members);
			return;
		}
		OCA.notification.onFail(
			t('circles', "The contact '{contact}' could not be added to the circle",
				{contact: result.display}) +
			': ' + ((result.error) ? result.error : t('circles', 'no error message')));
	},


	inviteMemberResult: function (result) {

		if (result.status === 1) {
			OCA.notification.onSuccess(
				t('circles', "The member '{name}' was invited to the circle",
					{name: result.display}));

			nav.displayMembers(result.members);
			return;
		}
		OCA.notification.onFail(
			t('circles', "The member '{name}' could not be invited to the circle",
				{name: result.display}) +
			': ' + ((result.error) ? result.error : t('circles', 'no error message')));
	},


	addMemberGroupResult: function (result) {

		if (result.user_type !== define.typeGroup) {

			return;
		}

		if (curr.circleDetails.type === define.typeClosed) {
			resultMembers.inviteMemberGroupResult(result);
			return;
		}

		if (result.status === 1) {
			OCA.notification.onSuccess(
				t('circles', "Members of the group '{name}' were added to the circle",
					{name: result.display}));

			nav.displayMembers(result.members);
			return;
		}
		OCA.notification.onFail(
			t('circles', "Members of the group '{name}' could not be added to the circle",
				{name: result.display}) +
			': ' +
			((result.error) ? result.error : t('circles', 'no error message')));
	},


	inviteMemberGroupResult: function (result) {

		if (result.status === 1) {
			OCA.notification.onSuccess(
				t('circles', "Members of the group '{name}' were invited to the circle",
					{name: result.display}));

			nav.displayMembers(result.members);
			return;
		}
		OCA.notification.onFail(
			t('circles', "Members of the group '{name}' could not be invited to the circle",
				{name: result.display}) +
			': ' +
			((result.error) ? result.error : t('circles', 'no error message')));
	},


	removeMemberResult: function (result) {
		if (result.status === 1) {

			elements.mainUIMembersTable.children("[member-id='" + result.user_id + "']").each(
				function () {
					if (Number($(this).attr('member-type')) === result.user_type) {
						$(this).hide(300);
					}
				});
			OCA.notification.onSuccess(
				t('circles', "The member '{name}' was removed from the circle",
					{name: result.display}));
			return;
		}

		OCA.notification.onFail(
			t('circles', "The member '{name}' could not be removed from the circle",
				{name: result.display}) +
			': ' +
			((result.error) ? result.error : t('circles', 'no error message')));
	},

	levelMemberResult: function (result) {
		if (result.status === 1) {
			OCA.notification.onSuccess(
				t('circles', "Member '{name}' updated",
					{name: result.display}));

			nav.displayMembers(result.members);
			return;
		}

		nav.displayMembers('');
		OCA.notification.onFail(
			t('circles', "The member '{name}' could not be updated", {name: result.display}) +
			': ' +
			((result.error) ? result.error : t('circles', 'no error message')));
	}

};


1			/*
2			* Circles - Bring cloud-users closer together.
3			*
4			* This file is licensed under the Affero General Public License version 3 or
5			* later. See the COPYING file.
6			*
7			* @author Maxence Lange <[email protected]>
8			* @copyright 2017
9			* @license GNU AGPL version 3 or any later version
10			*
11			* This program is free software: you can redistribute it and/or modify
12			* it under the terms of the GNU Affero General Public License as
13			* published by the Free Software Foundation, either version 3 of the
14			* License, or (at your option) any later version.
15			*
16			* This program is distributed in the hope that it will be useful,
17			* but WITHOUT ANY WARRANTY; without even the implied warranty of
18			* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
19			* GNU Affero General Public License for more details.
20			*
21			* You should have received a copy of the GNU Affero General Public License
22			* along with this program. If not, see <http://www.gnu.org/licenses/>.
23			*
24			*/
25
26			/** global: OC */
27			/** global: OCA */
28			/** global: Notyf */
29
30			/** global: actions */
31			/** global: nav */
32			/** global: elements */
33			/** global: curr */
34			/** global: api */
35
36
37			var resultMembers = {
38
39
40			generateItemResult: function (search, value) {
41
42			switch (value.type) {
			0 ignored issues – show Coding Style introduced 2017-08-25 10:04 UTC by Report Bug Copy Issue Report As per coding-style, `switch` statements should have a `default` case. Loading history...
43			case define.typeUser:
			0 ignored issues – show Bug introduced 2017-08-25 10:04 UTC by Report Bug Copy Issue Report The variable `define` seems to be never declared. If this is a global, consider adding a `/** global: define */` comment. This checks looks for references to variables that have not been declared. This is most likey a typographical error or a variable has been renamed. To learn more about declaring variables in Javascript, see the MDN. Loading history...
44			return resultMembers.generateItemUser(search, value);
45			case define.typeGroup:
46			return resultMembers.generateItemGroup(search, value);
47			case define.typeContact:
48			return resultMembers.generateItemContact(search, value);
49			}
			0 ignored issues – show Comprehensibility introduced 2017-08-25 10:04 UTC by Report Bug Copy Issue Report There is no `default` case in this switch, so nothing gets returned when all cases fail. You might want to consider adding a `default` or `return undefined` explicitly. Loading history...
50			},
51
52
53			enhanceSearchResult: function (search, display) {
54			display = escapeHTML(display);
55			if (search.length > 0) {
56			display = display.replace(new RegExp('(' + search + ')', 'gi'), '<b>$1</b>');
57			}
58
59			return display;
60			},
61
62
63			generateItemUser: function (search, value) {
64			return '<div class="result_top">' +
65			resultMembers.enhanceSearchResult(search, value.data.display) + '</div>' +
66			'<div class="result_bot">' + t('circles', 'Local User') + '</div>';
67			},
68
69			generateItemGroup: function (search, value) {
70			return '<div class="result_top">' +
71			resultMembers.enhanceSearchResult(search, value.data.display) + '</div>' +
72			'<div class="result_bot">' + t('circles', 'Local Group') + '</div>';
73			},
74
75			generateItemContact: function (search, value) {
76			var display = resultMembers.enhanceSearchResult(search, value.data.display);
77			var email = resultMembers.enhanceSearchResult(search, value.data.email);
78			var org = resultMembers.enhanceSearchResult(search, value.data.organization);
79			if (email !== '') {
80			email = ' - ' + email;
81			}
82
83			if (org !== '') {
84			display += ' (' + org + ')';
85			}
86
87			return '<div class="result_top">' + display + '</div>' +
88			'<div class="result_bot">' + t('circles', 'Contact') + email + '</div>';
89			},
90
91
92			searchMembersResult: function (response) {
93
94			elements.membersSearchResult.children().remove();
95
96			if (response === null) {
97			elements.membersSearchResult.fadeOut(0);
98			return;
99			}
100
101			var currSearch = response.search;
102			$.each(response.result, function (index, value) {
103			elements.membersSearchResult.append('<div class="members_search" data-type="' +
104			value.type + '" data-ident="' + escapeHTML(value.ident) + '">' +
105			resultMembers.generateItemResult(currSearch, value) + '</div>');
106			});
107
108			$('.members_search').on('click', function () {
109			var ident = $(this).attr('data-ident');
110			var type = $(this).attr('data-type');
111
112			if (Number(type) === define.typeGroup) {
			0 ignored issues – show Bug introduced 2017-08-25 10:04 UTC by Report Bug Copy Issue Report The variable `define` seems to be never declared. If this is a global, consider adding a `/** global: define */` comment. This checks looks for references to variables that have not been declared. This is most likey a typographical error or a variable has been renamed. To learn more about declaring variables in Javascript, see the MDN. Loading history...
113
114			OC.dialogs.confirm(
115			t('circles',
116			'This operation will add/invite all members of the group to the circle'),
117			t('circles', 'Please confirm'),
118			function (e) {
119			if (e === true) {
120			api.addMember(curr.circle, ident, type, resultMembers.addMemberResult);
121			}
122			});
123
124			} else {
125			api.addMember(curr.circle, ident, type, resultMembers.addMemberResult);
126			}
127			});
128
129			// elements.fillMembersSearch('users', response.ocs.data.exact.users,
130			// response.ocs.data.users); elements.fillMembersSearch('groups',
131			// response.ocs.data.exact.groups, response.ocs.data.groups); if
132			// (elements.membersSearchResult.children().length === 0) {
133			// elements.membersSearchResult.fadeOut(0); return; } $('.members_search').on('click',
134			// function () { curr.searchUserSelected = $(this).attr('searchresult'); if
135			// ($(this).attr('source') === 'groups') { OC.dialogs.confirm( t('circles', 'This
136			// operation will add/invite all members of the group to the circle'), t('circles', 'Please
137			// confirm'), function (e) { if (e === true) { api.addGroupMembers(curr.circle,
138			// curr.searchUserSelected, resultMembers.addGroupMembersResult); } }); } else {
139			// api.addMember(curr.circle, curr.searchUserSelected, resultMembers.addMemberResult); } });
140			elements.membersSearchResult.fadeIn(300);
141			},
142
143
144			addMemberResult: function (result) {
145
146			console.log(JSON.stringify(result));
			0 ignored issues – show Debugging Code introduced 2017-06-15 10:25 UTC by Report Bug Copy Issue Report `console.log` looks like debug code. Are you sure you do not want to remove it? Loading history...
147
148			resultMembers.addMemberUserResult(result);
149			resultMembers.addMemberGroupResult(result);
150			resultMembers.addMemberMailResult(result);
151			resultMembers.addMemberContactResult(result);
152			},
153
154
155			addMemberUserResult: function (result) {
156			if (result.user_type !== define.typeUser) {
			0 ignored issues – show Bug introduced 2017-08-25 10:04 UTC by Report Bug Copy Issue Report The variable `define` seems to be never declared. If this is a global, consider adding a `/** global: define */` comment. This checks looks for references to variables that have not been declared. This is most likey a typographical error or a variable has been renamed. To learn more about declaring variables in Javascript, see the MDN. Loading history...
157			return;
158			}
159
160			if (curr.circleDetails.type === define.typeClosed) {
161			resultMembers.inviteMemberResult(result);
162			return;
163			}
164
165			if (result.status === 1) {
166			OCA.notification.onSuccess(
167			t('circles', "The member '{name}' was added to the circle",
168			{name: result.display}));
169
170			nav.displayMembers(result.members);
171			return;
172			}
173
174			OCA.notification.onFail(
175			t('circles', "The member '{name}' could not be added to the circle",
176			{name: result.display}) +
177			': ' + ((result.error) ? result.error : t('circles', 'no error message')));
178			},
179
180
181			addMemberMailResult: function (result) {
182			if (result.user_type !== define.typeMail) {
			0 ignored issues – show Bug introduced 2017-08-25 10:04 UTC by Report Bug Copy Issue Report The variable `define` seems to be never declared. If this is a global, consider adding a `/** global: define */` comment. This checks looks for references to variables that have not been declared. This is most likey a typographical error or a variable has been renamed. To learn more about declaring variables in Javascript, see the MDN. Loading history...
183			return;
184			}
185
186			if (result.status === 1) {
187			OCA.notification.onSuccess(
188			t('circles', "The email address '{email}' was added to the circle",
189			{email: result.display}));
190
191			nav.displayMembers(result.members);
192			return;
193			}
194			OCA.notification.onFail(
195			t('circles', "The email address '{email}' could not be added to the circle",
196			{email: result.display}) +
197			': ' + ((result.error) ? result.error : t('circles', 'no error message')));
198			},
199
200			addMemberContactResult: function (result) {
201			if (result.user_type !== define.typeContact) {
			0 ignored issues – show Bug introduced 2017-08-25 10:04 UTC by Report Bug Copy Issue Report The variable `define` seems to be never declared. If this is a global, consider adding a `/** global: define */` comment. This checks looks for references to variables that have not been declared. This is most likey a typographical error or a variable has been renamed. To learn more about declaring variables in Javascript, see the MDN. Loading history...
202			return;
203			}
204
205			if (result.status === 1) {
206			OCA.notification.onSuccess(
207			t('circles', "The contact '{contact}' was added to the circle",
208			{contact: result.display}));
209
210			nav.displayMembers(result.members);
211			return;
212			}
213			OCA.notification.onFail(
214			t('circles', "The contact '{contact}' could not be added to the circle",
215			{contact: result.display}) +
216			': ' + ((result.error) ? result.error : t('circles', 'no error message')));
217			},
218
219
220			inviteMemberResult: function (result) {
221
222			if (result.status === 1) {
223			OCA.notification.onSuccess(
224			t('circles', "The member '{name}' was invited to the circle",
225			{name: result.display}));
226
227			nav.displayMembers(result.members);
228			return;
229			}
230			OCA.notification.onFail(
231			t('circles', "The member '{name}' could not be invited to the circle",
232			{name: result.display}) +
233			': ' + ((result.error) ? result.error : t('circles', 'no error message')));
234			},
235
236
237			addMemberGroupResult: function (result) {
238
239			if (result.user_type !== define.typeGroup) {
			0 ignored issues – show Bug introduced 2017-08-25 10:04 UTC by Report Bug Copy Issue Report The variable `define` seems to be never declared. If this is a global, consider adding a `/** global: define */` comment. This checks looks for references to variables that have not been declared. This is most likey a typographical error or a variable has been renamed. To learn more about declaring variables in Javascript, see the MDN. Loading history...
240			return;
241			}
242
243			if (curr.circleDetails.type === define.typeClosed) {
244			resultMembers.inviteMemberGroupResult(result);
245			return;
246			}
247
248			if (result.status === 1) {
249			OCA.notification.onSuccess(
250			t('circles', "Members of the group '{name}' were added to the circle",
251			{name: result.display}));
252
253			nav.displayMembers(result.members);
254			return;
255			}
256			OCA.notification.onFail(
257			t('circles', "Members of the group '{name}' could not be added to the circle",
258			{name: result.display}) +
259			': ' +
260			((result.error) ? result.error : t('circles', 'no error message')));
261			},
262
263
264			inviteMemberGroupResult: function (result) {
265
266			if (result.status === 1) {
267			OCA.notification.onSuccess(
268			t('circles', "Members of the group '{name}' were invited to the circle",
269			{name: result.display}));
270
271			nav.displayMembers(result.members);
272			return;
273			}
274			OCA.notification.onFail(
275			t('circles', "Members of the group '{name}' could not be invited to the circle",
276			{name: result.display}) +
277			': ' +
278			((result.error) ? result.error : t('circles', 'no error message')));
279			},
280
281
282			removeMemberResult: function (result) {
283			if (result.status === 1) {
284
285			elements.mainUIMembersTable.children("[member-id='" + result.user_id + "']").each(
286			function () {
287			if (Number($(this).attr('member-type')) === result.user_type) {
288			$(this).hide(300);
289			}
290			});
291			OCA.notification.onSuccess(
292			t('circles', "The member '{name}' was removed from the circle",
293			{name: result.display}));
294			return;
295			}
296
297			OCA.notification.onFail(
298			t('circles', "The member '{name}' could not be removed from the circle",
299			{name: result.display}) +
300			': ' +
301			((result.error) ? result.error : t('circles', 'no error message')));
302			},
303
304			levelMemberResult: function (result) {
305			if (result.status === 1) {
306			OCA.notification.onSuccess(
307			t('circles', "Member '{name}' updated",
308			{name: result.display}));
309
310			nav.displayMembers(result.members);
311			return;
312			}
313
314			nav.displayMembers('');
315			OCA.notification.onFail(
316			t('circles', "The member '{name}' could not be updated", {name: result.display}) +
317			': ' +
318			((result.error) ? result.error : t('circles', 'no error message')));
319			}
320
321			};
322

nextcloud / circles

Pull Request — master (#120)

resultMembers.addGroupMembersResult A

Complexity

Size

Duplication

Importance

1 Function

Duplication Side-by-Side

Filter issues like